(SP: 1) [PLATFORM] update CODEOWNERS, add missing packages, and adjus…#420
(SP: 1) [PLATFORM] update CODEOWNERS, add missing packages, and adjus…#420
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThree minor updates: added a new code owner, adjusted the default LinkedIn follower count, and bumped several TipTap-related frontend dependencies to 3.21.0. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
frontend/package.json (1)
40-43: Consider aligning all@tiptap/*package versions to a consistent minor release.The current mix of
^3.20.xand^3.21.xranges is being resolved by npm (e.g.,@tiptap/coreresolves to 3.21.0 despite^3.20.0in package.json), but standardizing to a single minor version across all@tiptappackages would avoid subtle version skew and simplify dependency maintenance. Consider updating the^3.20.xpackages (core,extension-code-block-lowlight,react,starter-kit) to^3.21.0for consistency, or audit whether all packages actually require3.21.xbefore aligning.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@frontend/package.json` around lines 40 - 43, The `@tiptap` packages in package.json are on mixed minor versions causing potential skew; update the entries for `@tiptap/core`, `@tiptap/extension-code-block-lowlight`, `@tiptap/react`, and `@tiptap/starter-kit` (the ones currently at ^3.20.x) to match ^3.21.0 (the same minor as `@tiptap/extension-image`, `@tiptap/extension-task-item`, `@tiptap/extension-task-list` and any other `@tiptap/`* deps), then run npm install and verify the app builds and tests pass; alternatively, if some packages require 3.20.x, perform a quick compatibility check and align all `@tiptap/`* entries to the same minor version accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@frontend/package.json`:
- Around line 40-43: The `@tiptap` packages in package.json are on mixed minor
versions causing potential skew; update the entries for `@tiptap/core`,
`@tiptap/extension-code-block-lowlight`, `@tiptap/react`, and `@tiptap/starter-kit`
(the ones currently at ^3.20.x) to match ^3.21.0 (the same minor as
`@tiptap/extension-image`, `@tiptap/extension-task-item`,
`@tiptap/extension-task-list` and any other `@tiptap/`* deps), then run npm install
and verify the app builds and tests pass; alternatively, if some packages
require 3.20.x, perform a quick compatibility check and align all `@tiptap/`*
entries to the same minor version accordingly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 9fb90661-808d-4569-8755-8c2aaea8cfee
⛔ Files ignored due to path filters (1)
frontend/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (3)
.github/CODEOWNERSfrontend/lib/about/stats.tsfrontend/package.json
Description
This PR fixes a production build blocker and updates related platform metadata/content.
It adds the missing TipTap dependencies required by the admin blog editor so
next buildno longer fails, updates CODEOWNERS, and refreshes the About section social-proof metric for LinkedIn.Related Issue
Issue: #<issue_number>
Changes
components/admin/blog/BlogTiptapEditor.tsxpackage.jsonandpackage-lock.jsonto reflect the new dependenciesCODEOWNERSfrontend/components/about/HeroSection.tsxDatabase Changes (if applicable)
How Has This Been Tested?
Additional notes:
Screenshots (if applicable)
Checklist
Before submitting
Reviewers
Summary by CodeRabbit